Skip to content

[CCOR-13383]fix:task definition form missing fields - #1586

Draft
najeebkp wants to merge 4 commits into
conductor-oss:mainfrom
najeebkp:CCOR-13383-task-definition-form-missing-fields
Draft

[CCOR-13383]fix:task definition form missing fields#1586
najeebkp wants to merge 4 commits into
conductor-oss:mainfrom
najeebkp:CCOR-13383-task-definition-form-missing-fields

Conversation

@najeebkp

Copy link
Copy Markdown
Contributor

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

CCOR-13383 Task Definitions Task tab is missing form controls for taskStatusListenerEnabled, maxRetryDelaySeconds, backoffJitterMs, totalTimeoutSeconds

  • Add form controls for task fields that were JSON-only

maxRetryDelaySeconds , backoffJitterMs , totalTimeoutSeconds and taskStatusListenerEnabled could only be edited as raw JSON. The retry cap and jitter apply to every retry policy, so unlike backoffScaleFactor they stay enabled for FIXED.

  • Stop blocking task definition save on an empty description

Save was disabled whenever the description was empty, with nothing on screen saying so, and the rule was skipped for new definitions — so a definition created without one could never be edited again. The API marks description optional. Both button wrappers now share one isSaveDisabled predicate.

Demo

Screen Recording 2026-08-28 at 5 51 38 PM

Enterprise UI Playwright Tests

Every PR automatically triggers the enterprise UI Playwright E2E test suite.
Tests run against conductor-ui main by default. To test against a different
conductor-ui branch, add this line anywhere in the PR description:

conductor-ui-branch: my-feature-branch

Comment thread ui-next/src/pages/definition/task/state/helpers.test.ts
@najeebkp
najeebkp marked this pull request as draft September 1, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants